types: fix various use of volatile in type registration

The use of volatile was incorrect in GLib and has been that way for
a long time. Recently however that has changed, and this makes GTK
follow suit to avoid using volatile in the type registration.

See also: https://gitlab.gnome.org/GNOME/glib/-/merge_requests/1719

Combined with the above merge request for GLib, this fixes a large
number of compilation warnings when using Clang.

Merge branch 'wayland-minimize' into 'master'

wayland: Implement minimization

Closes #2688

See merge request GNOME/gtk!2861

Merge branch 'constraint-editor-work' into 'master'

Constraint editor work

See merge request GNOME/gtk!2863

constraint-editor: Fix creating constant constraints

We were not making the button sensitive in the case
of a constant constraint, and we were not properly
creating constant constraints either.

constraint-editor: Fix saving of constraints

g_file_replace_contents take a gsize, so passing -1
for string length does not work here.

constraintlayout: Fix parsing of constant constraints

We were inadvertently turning constant constraints into
constraints against super, due to confusion between target
and source attribute.

Merge branch 'wip/chergert/gdk-macos-for-master' into 'master'

macos: fix rendering artifacts with hover transitions

See merge request GNOME/gtk!2862

constraint-editor: Improve display of constraints

No need to go for ALGOL60 style operators when we
have Unicode.

macos: fix rendering artifacts with hover transitions

Using an image surface seems to fix some rendering artifacts when
performing cross-fade CSS transitions.

wayland: Implement minimization

Use the set_minimized method of the xdg_toplevel
interface to implement minimization as well as possible.
It is not possible, since there is no corresponding
state that we could use to update our surface state,
but in practice, it works well enough.

Fixes: #2688

Merge branch 'wip/carlosg/for-master' into 'master'

gdk/wayland: Update to gtk_shell1 version 3

See merge request GNOME/gtk!2860

gdk/wayland: Update to gtk_shell1 version 3

Bring in line with gtk3, in terms of startup notification and
activation support.

Merge branch 'wip/chergert/gdk-macos-for-master' into 'master'

node-editor: fix length of text during save

See merge request GNOME/gtk!2859

Merge branch 'fix-doc-packtype' into 'master'

doc: Adapt GtkPackType documentation to GTK4 changes

See merge request GNOME/gtk!2858

node-editor: fix length of text during save

The length parameter for g_file_replace_contents() is a gsize, so -1 is
unsuitable here. Just use strlen() directly.

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

Closes #3278

See merge request GNOME/gtk!2856

Merge branch 'wip/chergert/gdk-macos-for-master' into 'master'

inspector: handle differences in DPI between backends

See merge request GNOME/gtk!2857

doc: Adapt GtkPackType documentation to GTK4 changes

Since GTK4, the functions that used the GtkPackType enumeration,
`gtk_box_pack_{start,end}`, were replaced by `gtk_box_{append,prepend}`,
hence this enumeration isn't used anymore by any function within the
GtkBox type, and the child packing properties were also removed for GTK4.
So this commit adapts the documentation accordingly.

Merge branch 'wip/otte/for-master' into 'master'

Wip/otte/for master

See merge request GNOME/gtk!2855

inspector: handle differences in DPI between backends

This fixes an issue on macOS where the text gets much larger once opening
the inspector.

inspector: Limit the width of readonly properties

For readonly properties, we show the serialized value
in a label. If we don't take precautions, this can cause
our window to grow extremely wide, and break things.
So, ellipsize things at a reasonable size.

Fixes: #3278

snapshot: Add cleanup function to snapshot state

Otherwise we're leaking data in error cases.

rendernode: Rename all gsk_render_node_peek_*() functions

Those are getters, they should be gsk_render_node_get_*() functions.

tests: Fix for new tranfer behavior of constructors

We were still unreffing a listmodel where the reference wasn't ours
anymore.

testsuite: Make defaultvalue test always get the property

Make the test not skip properties completely, just skip checking the
property.

This caught the last 2 commits.

textbuffer: Remove unused properties

The getters and setters were removed in
f53848c3606a9c46bcb3a0e568f36e573bdf84da but the pspecs were forgotten.

shortcut: Actions are objects

Merge branch 'wip/chergert/gdk-macos-for-master' into 'master'

macos: bring forward quartz search engine updates

See merge request GNOME/gtk!2854

Merge branch 'ebassi/inout-args' into 'master'

Remove pointless inout arguments

See merge request GNOME/gtk!2853

macos: bring forward quartz search engine updates

These were not cherry picked after GTK branched for 4 development.

docs: Add details about plain builds

Mention that build type 'plain' puts the responsibility
for debug features into the hands of the builder.

Related: !2831

Remove pointless inout arguments

GtkTreeView.get_tooltip_context() takes an inout X and Y coordinates,
but the "out" side is a side effect: the conversion from widget-relative
to bin window-relative coordinates is not documented, and can be done
using public API, if needed.

GtkIconView.get_tooltip_context() follows the same pattern, and takes
two inout arguments for the coordinates, but it does not change them any
more, after GtkIconView's bin window was dropped in commit 8dc5e13e.

There's really no point in having these `inout` arguments, and while
GtkTreeView and GtkIconView are certainly de-emphasised in GTK4, and we
nudge developers to move to the new list views, we should take advantage
of the API break to remove warts.

Merge branch 'ebassi/for-master' into 'master'

Introspection fixes

Closes #3281

See merge request GNOME/gtk!2852

Revert "Adwaita: avoid clipping menu/popover shadow"

There are big side effects to adding padding around all menus and popovers.

https://gitlab.gnome.org/GNOME/gtk/-/issues/1987#note_963889

This reverts commit d266bd62deaa5752ae79fe055696a9e1e411de2f.

Add type annotations for GtkShortcutAction types

The constructors and singleton getters need proper annotations for their
return type, given that we return a base type.

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!2851

Annotate GdkTimeCoord's array field

We need to resolve the array length manually, because g-ir-scanner
cannot turn an enumeration member into a constant size.

Annotate the return type for GtkTreeExpander.get_item()

We use gpointer for C convenience, but the returned value is really a
GObject.

Annotate out argument for gdk_popup_layout_get_offset()

Annotate the gsk_border_node_peek_widths() return value

We return a fixed sized C array of floats.

widget-factory: Remove an unused size group

This was left behind when 361407201b88c3aae2ee removed
the unused popover that this size group applied to.

docs: Fix the links for keybinding signals

GtkBindingSignal does not exist anymore, so link to
GtkSignalAction for an explanation of keybinding signals.

popover: Document signals

menubutton: Fix a gtk-doc complaint

gtk: Fix a doc oversight

gsk: Fix a doc oversight

Merge branch 'fix-gi-warning' into 'master'

Fix a warning caused by difference between prototype and definition

See merge request GNOME/gtk!2849

Merge branch 'wip/exalm/caption' into 'master'

widget-factory: Fix caption style class name

See merge request GNOME/gtk!2850

widget-factory: Fix caption style class name

It's lowercase.

Fix a warning caused by difference between prototype and definition

This commit fix the warning:

../gdk/wayland/gdkdisplay-wayland.c:1079: Warning: GdkWayland: gdk_wayland_display_set_cursor_theme: unknown parameter 'name' in documentation comment, should be 'theme'

Merge branch 'gsk-binding-api' into 'master'

gsk: Avoid using gtk css types in public api

Closes #2454

See merge request GNOME/gtk!2848

gsk: Avoid using gtk css types in public api

Using GtkCssSection in public headers here may be
ok from the C perspective, since it all ends up in
the same library anyway. But it causes circular
dependency problems for our gir files that are still
split by namespace.

To avoid this problem, copy the GtkCssLocation struct
struct as GskParseLocation, and pass take two of them
instead of a GtkCssSection in the error callback.

Update all users.

Fixes: #2454

Merge branch 'matthiasc/for-master' into 'master'

popover: Fix submenu navigation

Closes #3301

See merge request GNOME/gtk!2847

popover: Fix submenu navigation

In commit 024d832d943b2abf4f0d, we introduced a
cascade-popdown property that makes closing a submenu
propagate up and close its parent menus. This is the
behavior we want when a menuitem in the submen is
activated.

What we overlooked is that we still need to be able to
close a submenu during navigation, before opening another
one. And in this case, propagating the closing is breaking
things. Fix this by adding a private close_submenu api
to GtkPopoverMenu that avoids the propagation.

Fixes: #3301

build: Include GSK types in gtk_test_register_all_type()

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!2846

Merge branch 'ebassi/for-master' into 'master'

Ebassi/for master

See merge request GNOME/gtk!2840

widget-factory: Remove an unused popover

We are no longer using the hand-rolled variant of that
popover on page 3, so drop it from the ui file.

Merge branch 'wip/exalm/button' into 'master'

Allow pressing multiple buttons at once on touch

See merge request GNOME/gtk!2813

Merge branch 'wip/jimmac/popover-menu-submenus' into 'master'

Adwaita: fix up submenus again

See merge request GNOME/gtk!2845

Merge branch 'wip/chergert/for-master' into 'master'

gsk: use glFrameBufferTexture2D and release texture

See merge request GNOME/gtk!2842

Merge branch 'wip/jimmac/circular-button-focus' into 'master'

Adwaita: circular button focus

Closes #3368

See merge request GNOME/gtk!2844

Adwaita: fix up submenus again

- move menubar section under more generic popover.menu
  rather than resort to specificity dir(ltr) hacks

Adwaita: circular button focus

- have the same focus look like the other items

Fixes https://gitlab.gnome.org/GNOME/gtk/-/issues/3368

Merge branch 'wip/jimmac/spinbutton-spacing' into 'master'

Adwaita: spinbutton spacing

Closes #3370

See merge request GNOME/gtk!2841

a11y: Ensure valid object paths in the fallback code

When falling back to using the program name to create a unique base path
for the objects on the accessibility bus we need to ensure that the name
is a valid DBus object path.

gsk: use glFrameBufferTexture2D and release texture

glFrameBufferTexture maps to all faces of a cube and that is not needed
here. Additionally, texture_id is not deleted after we use the additional
flipped texture, but should be.

Adwaita: spinbutton spacing

Fixes https://gitlab.gnome.org/GNOME/gtk/-/issues/3370

a11y: Implement atspi.Cache

The AT-SPI cache interface is used to quickly populate the accessible
objects tree.

The tricky bit is ensuring that we emit change notifications on the
cache only when the cache is available, which means waiting until the
root is asynchronously registered.

a11y: Update the Cache interface introspection

The type for cached items is, unsurprisingly, wrong when compared to the
implementation inside atk-spi2-atk.

a11y: Add getters for GtkAtSpiContext

We are going to use them in order to implement atspi.Cache.

a11y: Use the tab widget to label the notebook stack page

a11y: Compute the base path in the root object

The root path is shared by all AtSpiContext instances, so we should
compute it once, instead of every time we instantiate a new context.

This allows us to defer the path creation at realization time and ensure
that we have a registered application.

a11y: Resync the AT-SPI XML

Mostly just changes in the annotations coming from at-spi2-core, but
it's good to try and keep the XML in sync.

Merge branch 'wip/jimmac/circular-button-padding' into 'master'

Adwaita: circular buttons

See merge request GNOME/gtk!2838

docs: Mention cairo-gobject as build requires

Using cairo from a system dependency and cairo-gobject
as subproject is problematic and left me stuck on a
hard-to-understand build error.

Adwaita: circular buttons

- the vertical padding was thown off with the chin/forehead of menus removed

Merge branch 'wip/jimmac/menu-clipped-shadow' into 'master'

Wip/jimmac/menu clipped shadow

Closes #1987

See merge request GNOME/gtk!2837

Adwaita: avoid clipping menu/popover shadow

Fixes https://gitlab.gnome.org/GNOME/gtk/-/issues/1987

gsk: Fix some docs oversights

These apis have been renamed a few times, and the docs
haven't kept up.

gdk: Docs cosmetics

Add a few missing parameters in doc comments.

docs: Fix a typo

docs: Remove gtk_render_insertion_cursor

This function was removed in 2bcef7f03058dac7cbac.

Adwaita: remove chin on menus

- clipping a rounded corner seems to be working fine for first and last item.

Addresses https://gitlab.gnome.org/GNOME/gtk/-/issues/3366

Merge branch 'wip/jimmac/subtle-sidebar' into 'master'

Adwaita: navigation sidebar color changes

Closes #3362

See merge request GNOME/gtk!2836

Adwaita: navigation sidebar color changes

- use more subtle selected state grey

Fixes https://gitlab.gnome.org/GNOME/gtk/-/issues/3362

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!2835

gtk-demo: Plug a memory leak

The Characters demo was leaking its columns.

modelbutton: Fix keynav for check and radio

The expected behavior when activating check or radio
menuitems via keynav is that Space toggles the item
but keeps the menu open, while Return toggles the
item and closes the menu.

Merge branch 'fix-tooltip-trigger' into 'master'

Fix tooltip triggering

See merge request GNOME/gtk!2833

Fix tooltip triggering

Ever since 5b5d2665d373c35f, tooltips don't work properly,
since we get more motion events now, and those keep the
tooltips from appearing. This can be seen e.g. on the icon
in the password entry in widget-factory, or on the section
switcher buttons in the Emoji chooser.

Ignore synthetic motion events for the purposes of
determining whether the user moved the mouse.

Merge branch 'widget-no-a11y-crash' into 'master'

gtkwidget: fix crashes with GTK_A11Y=none

Closes #3333

See merge request GNOME/gtk!2834

gtkwidget: fix crashes with GTK_A11Y=none

at_context can be NULL in that case, so guard against it.

Fixes #3333

Merge branch 'button-fill-var-removal' into 'master'

Adwaita: Stop using $button_fill

Closes #3354

See merge request GNOME/gtk!2828

Merge branch 'parse-debug-env-vars-always' into 'master'

Always parse GTK/GDK/GSK_DEBUG env vars and make some entries available in non-debug mode

See merge request GNOME/gtk!2830

Revert "Update comment annotations of gtk_tree_model_iter_previous and gtk_tree_model_iter_next to state that the ITER parameter is modified."

This reverts commit 1340ff2bc279329dc16b8d823100fec46b2f11d8.

Same as with 53cd499621b5

Always parse GTK/GDK/GSK_DEBUG env vars and make some entries available in non-debug mode

Currently GTK can be built with G_ENABLE_DEBUG which enables various debug code and parsing
of those env vars, or without, which instead of parsing them prints a warning if they are set.
While building with G_ENABLE_DEBUG isn't strictly needed it's the only way to make GTK_DEBUG=interactive work,
which is a nice thing to have always.

This enables parsing of those env vars in any case and allows specific values being marked as also
available when not built with G_ENABLE_DEBUG (interactive for example). If not built with G_ENABLE_DEBUG
then all unavailable values will be marked as such in the help output and a note is added that
GTK needs to be built with G_ENABLE_DEBUG to use them, which should help discoverability.

Update Ukrainian translation

Merge branch 'wip/carlosg/for-master' into 'master'

gtk/main: Reset active state on grab broken only if implicit

See merge request GNOME/gtk!2832

gtk/main: Reset active state on grab broken only if implicit

An implicit grab means some button is down, reset the active state
only in that case when we get a grab broken event.

Avoids active state accounting warnings when we do get active grabs
broken (e.g. after selecting a menu option).